Skip to content

dhi: enterprise get-started wording fix#24592

Merged
craig-osterhout merged 1 commit intodocker:mainfrom
craig-osterhout:dhi-entguide-fix
Apr 1, 2026
Merged

dhi: enterprise get-started wording fix#24592
craig-osterhout merged 1 commit intodocker:mainfrom
craig-osterhout:dhi-entguide-fix

Conversation

@craig-osterhout
Copy link
Copy Markdown
Contributor

Description

Clarified the prerequisites structure in the DHI Enterprise guide. The previous wording incorrectly listed Docker Desktop 4.65+ as part of "one of the following" options, making it appear optional or mutually exclusive with the subscription/trial requirement.

Changed:

  • Restructured prerequisites to show Docker Desktop 4.65+ is always required (not optional) for CLI
    Clarified that "one of the following" applies only to the subscription/trial choice
  • Made it clear there are 3 distinct prerequisites:
    • Organization owner access (required)
    • DHI Select/Enterprise subscription OR trial (required - pick one)
    • Docker Desktop 4.65+ (required for cli)

Related issues or tickets

Reviews

  • Editorial review

Signed-off-by: Craig Osterhout <craig.osterhout@docker.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 31, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 2e1f7ff
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69cc058ee200a300086e9a00
😎 Deploy Preview https://deploy-preview-24592--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@craig-osterhout
Copy link
Copy Markdown
Contributor Author

/review

Copy link
Copy Markdown
Contributor

@docker-agent docker-agent Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

This PR successfully clarifies the prerequisites structure for the DHI Enterprise guide. The changes improve clarity by:

  1. Restructuring the prerequisites into a clear bulleted list format
  2. Separating the requirements to show that:
    • Organization owner access is always required
    • One of subscription/trial is required (not both)
    • Docker Desktop 4.65+ is always required for CLI usage

The new structure eliminates the ambiguity where Docker Desktop appeared to be part of the "one of the following" choice. The list formatting is correct, parallel in structure, and follows the Docker documentation style guide.

No issues found in the changed code.

@craig-osterhout craig-osterhout requested a review from a team March 31, 2026 17:40
@craig-osterhout craig-osterhout added status/review Pull requests that are ready for review area/dhi labels Mar 31, 2026
@craig-osterhout craig-osterhout enabled auto-merge (squash) March 31, 2026 22:35
Copy link
Copy Markdown
Contributor

@dvdksn dvdksn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@craig-osterhout craig-osterhout merged commit d64c51f into docker:main Apr 1, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/dhi status/review Pull requests that are ready for review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants